Skip to content

Add OADP single-stack IPv6 cluster provisioning workflow on Equinix M…#81210

Draft
PrasadJoshi12 wants to merge 2 commits into
openshift:mainfrom
PrasadJoshi12:oadp-baremetalds-ipv6-workflow
Draft

Add OADP single-stack IPv6 cluster provisioning workflow on Equinix M…#81210
PrasadJoshi12 wants to merge 2 commits into
openshift:mainfrom
PrasadJoshi12:oadp-baremetalds-ipv6-workflow

Conversation

@PrasadJoshi12

@PrasadJoshi12 PrasadJoshi12 commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Add OADP single-stack IPv6 cluster provisioning workflow on Equinix Metal

Summary by CodeRabbit

This PR updates the OpenShift OADP CI infrastructure to provision a single-stack IPv6 bare-metal cluster on Equinix Metal for OADP E2E QE. It introduces a new IPv6-specific step-registry workflow and wires it into the release-v1.5 testing variant configuration.

Key changes:

  • New step-registry workflow: oadp-baremetalds-ipv6-oadp-qe

    • Targets the equinix-ocp-metal cluster profile.
    • Configures dev-scripts for OVN-Kubernetes single-stack IPv6 provisioning via DEVSCRIPTS_CONFIG (IP_STACK=v6, NETWORK_TYPE=OVNKubernetes).
    • Sets DEBUG_WAIT_TIMEOUT to 43200.
    • Runs through the standard chain: baremetalds-ofcir-pre → test step (oadp-debug-wait) → baremetalds-ofcir-post.
    • Includes embedded documentation of the IPv6 provisioning scenario.
  • New release/variant CI configuration: ci-operator/.../oadp-qe-oadp-e2e-qe-release-v1.5__ipv6-cluster.yaml

    • Binds base_images.dev-scripts to ocp-kni:dev-scripts.
    • Sets build_root.image_stream_tag to rhel-9-release-golang-1.19-openshift-4.15.
    • Configures releases.latest.integration for product ocp version 5.0.
    • Applies default resource settings (e.g., cpu: 100m, memory: 200Mi, limits.memory: 6Gi).
    • Adds a new optional test entry oadp-baremetalds-ipv6 (always_run: false) that triggers the new workflow (oadp-baremetalds-ipv6-oadp-qe).
    • Links metadata to branch: release-v1.5, org: oadp-qe, repo: oadp-e2e-qe, variant: ipv6-cluster.
  • Ownership/metadata: Adds workflow and artifact ownership metadata for the new IPv6 workflow (including prasadjoshi12 as approver/reviewer).

@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 29, 2026
@openshift-ci

openshift-ci Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@coderabbitai

coderabbitai Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

  • @coderabbitai resume to resume automatic reviews.
  • @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

  • ▶️ Resume reviews
  • 🔍 Trigger review

Walkthrough

Adds a new OADP E2E QE CI job for a release-v1.5 IPv6 bare-metal cluster, with matching workflow definition, workflow ownership metadata, and ci-operator job configuration.

Changes

OADP IPv6 bare-metal CI wiring

Layer / File(s) Summary
Workflow definition and ownership
ci-operator/step-registry/oadp/baremetalds/ipv6-oadp-qe/oadp-baremetalds-ipv6-oadp-qe-workflow.yaml, ci-operator/step-registry/oadp/baremetalds/ipv6-oadp-qe/oadp-baremetalds-ipv6-oadp-qe-workflow.metadata.json, ci-operator/step-registry/oadp/baremetalds/ipv6-oadp-qe/OWNERS
Defines the oadp-baremetalds-ipv6-oadp-qe workflow, its IPv6 provisioning settings, and its approver/reviewer ownership.
Job registration
ci-operator/config/oadp-qe/oadp-e2e-qe/oadp-qe-oadp-e2e-qe-release-v1.5__ipv6-cluster.yaml
Registers the optional oadp-baremetalds-ipv6 test for the ipv6-cluster variant and ties it to the workflow.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested labels

rehearsals-ack

🚥 Pre-merge checks | ✅ 15
✅ Passed checks (15 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: adding an OADP single-stack IPv6 provisioning workflow on Equinix Metal.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Stable And Deterministic Test Names ✅ Passed No Ginkgo test titles were added; the PR only introduces static ci-operator/workflow YAML and metadata, with no dynamic names in test titles.
Test Structure And Quality ✅ Passed PR only adds ci-operator/OWNERS YAML config; no Ginkgo test code or assertions are present to review.
Microshift Test Compatibility ✅ Passed No Ginkgo test code was added; the PR only adds ci-operator/workflow configs, with no MicroShift-sensitive APIs, tags, or guards present.
Single Node Openshift (Sno) Test Compatibility ✅ Passed PASS: This PR only adds ci-operator/workflow config and a debug-wait step; no new Ginkgo test bodies or SNO-relevant node-topology assumptions are introduced.
Topology-Aware Scheduling Compatibility ✅ Passed The new files only add CI workflow/config plumbing (cluster profile/env/pre/test/post) and no pod scheduling rules, replicas, affinities, nodeSelectors, or PDBs.
Ote Binary Stdout Contract ✅ Passed PR only adds ci-operator/workflow YAML and OWNERS metadata; no Go process-level code or stdout writes were added.
Ipv6 And Disconnected Network Test Compatibility ✅ Passed No new Ginkgo test code was added; the workflow provisions a single-stack IPv6 cluster and runs only the internal oadp-debug-wait sleep step, with no public-internet or IPv4-only assumptions.
No-Weak-Crypto ✅ Passed The new CI config/workflow/OWNERS files contain no crypto code, weak algorithms, or secret/token comparisons.
Container-Privileges ✅ Passed Changed YAMLs only add workflow/env wiring; targeted searches found no privileged, hostPID/Network/IPC, SYS_ADMIN, allowPrivilegeEscalation, or securityContext flags.
No-Sensitive-Data-In-Logs ✅ Passed Reviewed the added workflow/config files; they only add cluster provisioning settings and an OWNERS entry, with no passwords, tokens, keys, PII, or customer data.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@PrasadJoshi12

Copy link
Copy Markdown
Contributor Author

/pj-rehearse pull-ci-oadp-qe-oadp-e2e-qe-release-v1.5-ipv6-cluster-oadp-baremetalds-ipv6

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@PrasadJoshi12: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci

openshift-ci Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: PrasadJoshi12
Once this PR has been reviewed and has the lgtm label, please assign dfrazzette for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@PrasadJoshi12 PrasadJoshi12 force-pushed the oadp-baremetalds-ipv6-workflow branch from f6dfda8 to 3bdab07 Compare June 29, 2026 14:58
@PrasadJoshi12

Copy link
Copy Markdown
Contributor Author

/pj-rehearse pull-ci-oadp-qe-oadp-e2e-qe-release-v1.5-ipv6-cluster-oadp-baremetalds-ipv6

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@PrasadJoshi12: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@PrasadJoshi12 PrasadJoshi12 force-pushed the oadp-baremetalds-ipv6-workflow branch from 3bdab07 to 85f6443 Compare June 29, 2026 15:23
@PrasadJoshi12

Copy link
Copy Markdown
Contributor Author

/pj-rehearse pull-ci-oadp-qe-oadp-e2e-qe-release-v1.5-ipv6-cluster-oadp-baremetalds-ipv6

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@PrasadJoshi12: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@PrasadJoshi12

Copy link
Copy Markdown
Contributor Author

/pj-rehearse pull-ci-oadp-qe-oadp-e2e-qe-release-v1.5-ipv6-cluster-oadp-baremetalds-ipv6

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@PrasadJoshi12: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@PrasadJoshi12 PrasadJoshi12 force-pushed the oadp-baremetalds-ipv6-workflow branch from 85f6443 to 7c5f4fb Compare June 29, 2026 17:45
…etal

Adds a new workflow oadp-baremetalds-ipv6-oadp-qe that provisions a
single-stack IPv6 bare metal cluster on Equinix Metal using dev-scripts
with IP_STACK=v6. Cluster stays alive for 12 hours via oadp-debug-wait.

Also adds a CI config variant (release-v1.5__ipv6-cluster) with a
presubmit job that can be triggered via:
  /test ipv6-cluster-oadp-baremetalds-ipv6
or via Gangway:
  pull-ci-oadp-qe-oadp-e2e-qe-release-v1.5-ipv6-cluster-oadp-baremetalds-ipv6

Co-authored-by: Cursor <cursoragent@cursor.com>
@PrasadJoshi12 PrasadJoshi12 force-pushed the oadp-baremetalds-ipv6-workflow branch from 7c5f4fb to ff331f8 Compare June 29, 2026 18:07
@PrasadJoshi12

Copy link
Copy Markdown
Contributor Author

/pj-rehearse pull-ci-oadp-qe-oadp-e2e-qe-release-v1.5-ipv6-cluster-oadp-baremetalds-ipv6

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@PrasadJoshi12: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

@openshift-ci

openshift-ci Bot commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

@PrasadJoshi12: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/rehearse/oadp-qe/oadp-e2e-qe/release-v1.5/ipv6-cluster-oadp-baremetalds-ipv6 ff331f8 link unknown /pj-rehearse pull-ci-oadp-qe-oadp-e2e-qe-release-v1.5-ipv6-cluster-oadp-baremetalds-ipv6

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

…d timeouts

- Add capabilities/intranet and 14h timeout to ipv6-cluster config
- Reduce DEBUG_WAIT_TIMEOUT from 12h to 10h in workflow
- Expand OWNERS with oadp-e2e-qe approvers/reviewers

Co-authored-by: Cursor <cursoragent@cursor.com>
@openshift-merge-bot

Copy link
Copy Markdown
Contributor

[REHEARSALNOTIFIER]
@PrasadJoshi12: the pj-rehearse plugin accommodates running rehearsal tests for the changes in this PR. Expand 'Interacting with pj-rehearse' for usage details. The following rehearsable tests have been affected by this change:

Test name Repo Type Reason
pull-ci-oadp-qe-oadp-e2e-qe-release-v1.5-ipv6-cluster-oadp-baremetalds-ipv6 oadp-qe/oadp-e2e-qe presubmit Presubmit changed
Interacting with pj-rehearse

Comment: /pj-rehearse to run up to 5 rehearsals
Comment: /pj-rehearse skip to opt-out of rehearsals
Comment: /pj-rehearse {test-name}, with each test separated by a space, to run one or more specific rehearsals
Comment: /pj-rehearse more to run up to 10 rehearsals
Comment: /pj-rehearse max to run up to 25 rehearsals
Comment: /pj-rehearse auto-ack to run up to 5 rehearsals, and add the rehearsals-ack label on success
Comment: /pj-rehearse list to get an up-to-date list of affected jobs
Comment: /pj-rehearse abort to abort all active rehearsals
Comment: /pj-rehearse network-access-allowed to allow rehearsals of tests that have the restrict_network_access field set to false. This must be executed by an openshift org member who is not the PR author

Once you are satisfied with the results of the rehearsals, comment: /pj-rehearse ack to unblock merge. When the rehearsals-ack label is present on your PR, merge will no longer be blocked by rehearsals.
If you would like the rehearsals-ack label removed, comment: /pj-rehearse reject to re-block merging.

@PrasadJoshi12

Copy link
Copy Markdown
Contributor Author

/pj-rehearse pull-ci-oadp-qe-oadp-e2e-qe-release-v1.5-ipv6-cluster-oadp-baremetalds-ipv6

@openshift-merge-bot

Copy link
Copy Markdown
Contributor

@PrasadJoshi12: now processing your pj-rehearse request. Please allow up to 10 minutes for jobs to trigger or cancel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant